-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
#81 Updated README #93
#81 Updated README #93
Conversation
1088e69
to
b174d8d
Compare
7a0a5f8
to
1be4af0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for doing this @michaelryanpeter! 🙏🏽
@anik120 Would you PTAL at the updates and let know if they address your concerns? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking GREAT! Other than the missing example outputs:
/lgtm
Also the DCO check is failing (if you click on |
bef874d
to
18b6c37
Compare
New changes are detected. LGTM label has been removed. |
18b6c37
to
3cfb851
Compare
@anik120 and @everettraven: I got kind working, added example outputs, and squashed and signed my commit. Do I need to request more reviews in Slack? |
e8b1577
to
ef5090d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall looks great!
@michaelryanpeter if you want some extra eyes it doesn't hurt to throw it in the #olm-dev channel in the Kubernetes slack |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/approve
We can get this merged after the rebase
Issue: #81
Update the catalogd README to reflect the current state of the project and its place in the OLM v1 project.